-
Notifications
You must be signed in to change notification settings - Fork 159
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Some cleanup in Thrust config headers #1934
Conversation
I see this errors in the CI:
But I fail to find the file |
This is from our thirdparty RAPIDS build in CI. RAPIDS applies some patches to Thrust source files as part of their build. Some of the changes here must conflict with one of their patch files. We don't need to block merging this PR based on this failure, but we should let @robertmaynard @trxcllnt @bdice know so they can update the necessary patch files. |
https://github.com/rapidsai/rapids-cmake/blob/branch-24.08/rapids-cmake/cpm/patches/cccl/revert_pr_211.diff is the change we are applying. I don't see this PR changing |
@robertmaynard thank you for the answer! I am surprised this was not caught by the CI for the offending PR. In any case, please update your patch file then! Thx! |
The rapids-cmake patch file can't be naively updated as we pull a specific SHA1 and that would cause the patch to fail with our SHA1. That better option is to have the rapids ci jobs in CCCL updated to use an updated patch file ( https://github.com/NVIDIA/cccl/blob/main/ci/rapids/post-create-command.sh#L54 ) |
🟨 CI finished in 5h 21m: Pass: 98%/249 | Total: 5d 00h | Avg: 28m 55s | Max: 1h 04m | Hits: 42%/245883
|
Project | |
---|---|
CCCL Infrastructure | |
libcu++ | |
CUB | |
+/- | Thrust |
CUDA Experimental |
Modifications in project or dependencies?
Project | |
---|---|
CCCL Infrastructure | |
libcu++ | |
+/- | CUB |
+/- | Thrust |
CUDA Experimental |
🏃 Runner counts (total jobs: 249)
# | Runner |
---|---|
178 | linux-amd64-cpu16 |
40 | linux-amd64-gpu-v100-latest-1 |
16 | linux-arm64-cpu16 |
15 | windows-amd64-cpu16 |
🟩 CI finished in 6h 30m: Pass: 100%/249 | Total: 5d 01h | Avg: 29m 12s | Max: 1h 04m | Hits: 43%/248439
|
Project | |
---|---|
CCCL Infrastructure | |
libcu++ | |
CUB | |
+/- | Thrust |
CUDA Experimental |
Modifications in project or dependencies?
Project | |
---|---|
CCCL Infrastructure | |
libcu++ | |
+/- | CUB |
+/- | Thrust |
CUDA Experimental |
🏃 Runner counts (total jobs: 249)
# | Runner |
---|---|
178 | linux-amd64-cpu16 |
40 | linux-amd64-gpu-v100-latest-1 |
16 | linux-arm64-cpu16 |
15 | windows-amd64-cpu16 |
This PR proses some cleanup in the Thrust config headers:
THRUST_TRAILING_RETURN